Skip to content

Conversation

@ChronosSF
Copy link
Member

@ChronosSF ChronosSF commented Jul 14, 2025

Closes #16000
Closes #15837

Additional information (check all that apply):

  • Bug fix
  • New functionality
  • Documentation
  • Demos
  • CI/CD

Checklist:

  • All relevant tags have been applied to this PR
  • This PR includes unit tests covering all the new code (test guidelines)
  • This PR includes API docs for newly added methods/properties (api docs guidelines)
  • This PR includes feature/README.MD updates for the feature docs
  • This PR includes general feature table updates in the root README.MD
  • This PR includes CHANGELOG.MD updates for newly added functionality
  • This PR contains breaking changes
  • This PR includes ng update migrations for the breaking changes (migrations guidelines)
  • This PR includes behavioral changes and the feature specification has been updated with them

@ChronosSF ChronosSF requested a review from rkaraivanov July 25, 2025 15:21
@ChronosSF ChronosSF added ♿ a11y When the issue or PR is related to accessibility 📅 date-picker combo simple-combo version: 20.0.x labels Jul 25, 2025
@ChronosSF ChronosSF changed the title Sstoychev/bypassing input required logic 20 Combo/SimpleCombo/DatePicker aria-required properties addition - 20.0.x Jul 25, 2025
@ChronosSF ChronosSF marked this pull request as ready for review July 25, 2025 15:23
@ChronosSF ChronosSF requested a review from IvanKitanov17 July 25, 2025 15:23
<ng-content select="igx-hint, [igxHint]"></ng-content>
</ng-container>
<input igxInput #comboInput name="comboInput" type="text" [value]="displayValue" readonly
[required]="required" [attr.aria-required]="required"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
[required]="required" [attr.aria-required]="required"
[required]="required"

IMO this is clutter, since the required attribute on a native input element semantically does the same thing.

</ng-container>

<input #comboInput igxInput [value]="displayValue" role="combobox"
[required]="required" [attr.aria-required]="required" [externalValidate]="externalValidate"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See previous comment

@ChronosSF
Copy link
Member Author

20.0.x received its last patch and will no longer be supported.

@ChronosSF ChronosSF closed this Sep 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

combo 📅 date-picker simple-combo version: 20.0.x ♿ a11y When the issue or PR is related to accessibility

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants